Skip to content

Conversation

@Zaimwa9
Copy link
Contributor

@Zaimwa9 Zaimwa9 commented Oct 20, 2025

Thanks for submitting a PR! Please check the boxes below:

  • I have added information to docs/ if required so people know about the feature!
  • I have filled in the "Changes" section below?
  • I have filled in the "How did you test this code" section below?
  • I have used a Conventional Commit title for this Pull Request

Changes

  • new utility getUserAgent to get version from composer as flagsmith-ruby-sdk/v4.3.0 or returns flagsmith-php-sdk/unknown (not realistic as the version is required in Gem)
  • Added header in API and Analytics requests

How did you test this code?

  • Added tests
  • used Dummy script and got the desired
Headers configured in Faraday connection:
  Accept: application/json
  Content-Type: application/json
  X-Environment-Key: test-key
  User-Agent: flagsmith-ruby-sdk/4.3.0

@Zaimwa9 Zaimwa9 requested a review from a team as a code owner October 20, 2025 16:46
@Zaimwa9 Zaimwa9 requested review from gagantrivedi and removed request for a team October 20, 2025 16:46
@Zaimwa9 Zaimwa9 merged commit a7216f5 into main Oct 21, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants